Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rebase #95

Merged
merged 12 commits into from
Feb 7, 2024
Merged

Rebase #95

merged 12 commits into from
Feb 7, 2024

Conversation

tylerlight071
Copy link
Owner

No description provided.

- Made animal table be shown when exiting add animal screen
- Moved register new user into user management and added Audit Log function
- Created a new script for client database which for now only allows the user to go through the adoption form process
- Removed """ """ comments as they looked untidy compared to the rest of my comments
- Added exit function to the updating animal entries and cleaned up when an animal is not found
- Moved ADMIN dashboard around so now reset password us in update user info
- Made new function for modifying the animal database much like the client database however this has full functionality much like before
- Removed search via index as decided that was not necessary as this is a form to match customers to potential pets
- Extended the form with different section headers + section ends and extra fields to fill out
- Extended dictionary to reflect and save these new fields
-  Changed function name to 'adopt_animal_form'
- Changed file name to customer_adoption_form_dog to reflect the future of 2+ forms
- Created new function called get_input that is a simple and easiesr way to check if a field is empty without restarting the entire form like before
@tylerlight071 tylerlight071 self-assigned this Feb 7, 2024
@tylerlight071 tylerlight071 merged commit 0dbf8ce into Bug-Fixes Feb 7, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant